Add a note to the style guide about abbreviations - #1844
Conversation
resolves python#1824 After the section on specific terms, but before "simple language", add a section that explains that documentation should spell out acronyms, preferring the "<full-spelling> (<acronym>)" format.
Documentation build overview
28 files changed ·
|
|
Hi Stephen, this issue was originally specific to
|
|
Oh, I hadn't even thought to put it in the role usage docs! 🤦 I want to keep some explanation of the rationale. I'll move things as you suggest, and I think the "some assistive technology" note will end up with the roles. Edit done! LMK if it needs more tuning. |
The role is documented in with other sphinx roles. In order for the rst to read easily, one line of non-semantic whitespace was added to a list. The acronym usage note is moved to the end of "Use simple language"
Maybe... I like "Use simple language". It has two virtues:
I could probably be convinced, but we can discuss outside of this PR. 😁 |
|
As I said, that's for another day ;-) I also have to catch up on the Discord discussion. Thanks for the changes, it's what I had in mind. |
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Co-authored-by: Stan Ulbrych <stan@python.org>
| Don't use Latin abbreviations like "e.g." or "i.e." where English words will do, | ||
| such as "for example" or "that is." | ||
|
|
||
| In general, the first time an acronym is used on a page, spell it out. |
There was a problem hiding this comment.
Ironically nothing in the devguide defines the "HTML" abbreviation which you use in the other part of this PR. And I think that's a good thing: in computing there are some terms that are more widely understood as abbreviations than as the full form, and adding the full form is more likely to distract than to help readers. HTML is one of those terms. Other examples off the top of my head are UTF-8, PNG, SVG, SQL.
There was a problem hiding this comment.
Yeah, sometimes an acronym becomes a term or word in its own right.
The guidance should be read as flexible in this respect.
We added the "In general" to try to succinctly cover this. Do you feel it needs more explanation? We can try another arrangement if the current text looks too strong.
There was a problem hiding this comment.
The current text will likely lead people to comment that you need to say "Hyper-Text Markup Language". I'd suggest wording like "Commonly understood acronyms such as HTML or UTF-8 do not need to be expanded."
There was a problem hiding this comment.
I tried a few different "spellings" of the idea, and ended up with almost exactly this suggested text.
(Sorry, I broke the typical rules and did an amend, as I forgot to put Jelle in as a co-author at first!)
There was a problem hiding this comment.
as I forgot to put Jelle in as a co-author at first!)
In that case, in the future please comment for whoever is merging to add it in (we squash anyway), to avoid amends.
There was a problem hiding this comment.
Or add an empty commit with the co-author, or don't worry about it :)
There was a problem hiding this comment.
I'll use an empty commit if it comes up again! That way, it's automatically included in the squash and nobody has to do anything extra.
I strive for perfection, including in my citations. Clearly I need to take the no-force-pushes rule into account more strongly! 😅
Co-authored-by: Jelle Zijlstra <906600+JelleZijlstra@users.noreply.github.com>
a82602e to
d0cdf9e
Compare
StanFromIreland
left a comment
There was a problem hiding this comment.
A couple final nits, otherwise looks great.
Additionally, can you fix these, please:
$ gg ":abbr:"
developer-workflow/development-cycle.rst:to the :abbr:`VCS (version control system)`.
developer-workflow/stdlib.rst:tracker and :abbr:`VCS (version control system)`.
developer-workflow/stdlib.rst:by following the :abbr:`PEP (Python Enhancement Proposal)` process.
development-tools/clinic/index.rst:The Argument Clinic :abbr:`CLI (Command-Line Interface)` is typically used to
getting-started/pull-request-lifecycle.rst::abbr:`VCS (version control system)` to be released
getting-started/setup-building.rst: Contains the :abbr:`PEG (Parser Expression Grammar)` grammar file for
testing/coverage.rst:to the language and :abbr:`stdlib (standard library)` are accompanied by
With those, LGTM :-)
| interpreted text roles of the form ``:rolename:`content``` | ||
| to insert semantic markup in documents. | ||
|
|
||
| In the CPython documentation, there are a couple common cases |
There was a problem hiding this comment.
| In the CPython documentation, there are a few common cases |
We're now at three :-)
There was a problem hiding this comment.
Hmm, that suggestion seems broken, for clarity: "couple" -> "few"
There was a problem hiding this comment.
I think I'll do a normal commit with this + the :abbr: fixes + the fix below, so I'll try not to think about what that weird rendering in GitHub means.
| Don't use Latin abbreviations like "e.g." or "i.e." where English words will do, | ||
| such as "for example" or "that is." | ||
|
|
||
| In general, the first time an acronym is used on a page, spell it out. |
There was a problem hiding this comment.
Can we also add "abbreviation" here please.
Hat tip to @hugovk for this! Not only the issue (#1824), but also his comments preceding it made this easy to write.
This as a draft because it conflicts with #1828, which is already approved and should ideally merge first.
Once that's done, the exact positioning of this content in the page may change.
Open question: should this be converted to be a subsection of "Use simple language" or "Specific words"?
resolves #1824
After the section on specific terms, but before "simple language", add a section that explains that documentation should spell out acronyms, preferring the
"<full-spelling> (<acronym>)"format.